Skip to content

UI changes#243

Closed
Lalitya31 wants to merge 12 commits into
komalharshita:mainfrom
Lalitya31:ui
Closed

UI changes#243
Lalitya31 wants to merge 12 commits into
komalharshita:mainfrom
Lalitya31:ui

Conversation

@Lalitya31

Copy link
Copy Markdown
Contributor

Summary

Improvised :

  1. Navbar
  2. Subtle out shade
  3. Even out colors
  4. Remove unmatched color themes while keeping the original unchanged
  5. Reduced pill structure
  6. Introduced creative ways of info presentation like terminals , charts
  7. Changed typography from generic system to Manrope system

Related Issue

Closes #44

Type of Change [required]

  • Bug fix — resolves a broken behaviour
  • Feature — adds new functionality
  • Data — adds new projects to data/projects.json
  • Documentation — updates docs, README, or code comments only
  • Style — CSS or visual changes only, no logic change
  • Refactor — restructures code without changing behaviour
  • Test — adds or updates tests

Screenshots (if UI change)

Before :
image
image
image

AFTER :
image
image
image
image

How to Test This PR [required]

  1. Clone this branch: git checkout ui
  2. Install dependencies: pip install -r requirements.txt
  3. Run the app: python3 -m http.server 800
  4. Open http://localhost:8000/templates/index.html a
Before After
screenshot screenshot

Self-Review Checklist [required]

  • I have read CONTRIBUTING.md and followed all guidelines
  • My branch name follows the convention: feat/, fix/, docs/, data/, style/, test/
  • I have run python tests/test_basic.py and all 27 tests pass
  • I have run flake8 . locally and there are no errors
  • I have not introduced any print() or console.log() debug statements
  • Every new function I wrote has a docstring
  • I have not modified files outside the scope of the linked issue
  • If I changed the UI, I tested it at 375px (mobile) and 1280px (desktop)
  • If I added a project to the dataset, it has all required JSON fields

Copilot AI review requested due to automatic review settings May 17, 2026 15:02
@vercel

vercel Bot commented May 17, 2026

Copy link
Copy Markdown

@Lalitya31 is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@Lalitya31

Copy link
Copy Markdown
Contributor Author

@komalharshita resolved merge conflicts , ready for review

@komalharshita

Copy link
Copy Markdown
Owner

@Lalitya31 Thanks for the detailed UI contribution and the effort put into improving the visual hierarchy and overall polish of the project. There are several thoughtful ideas here, especially around spacing, typography, and information presentation.

However, this PR is not ready for approval yet for a few reasons:

  • The branch currently has unresolved merge conflicts (templates/index.html) that must be resolved first.
  • The PR scope is extremely broad and combines many unrelated UI redesign decisions into a single change set (navbar redesign, typography overhaul, spacing system, color system, terminal-style cards, separators, etc.).
  • Some visual changes significantly alter the existing design language without a clearly defined design system or issue-specific scope.
  • The large number of UI/style modifications makes regression review and maintainability difficult.

Please:

  • resolve all merge conflicts
  • clean/reduce unnecessary UI rewrites where possible
  • keep the PR more tightly scoped to the linked issue

Once conflicts are resolved and the scope is refined, this can be reviewed again.

@Lalitya31

Copy link
Copy Markdown
Contributor Author

@komalharshita I have resolved conflicts twice, yes I will do it again.

Since perspective varies I cannot decide what changes are "unnecessary" because from my experience of design all of them seemed relevant to me for being a balanced visual identity (yes I do follow UX philosophies a bit rigidly sometimes) so could you please specify what changes do I need to revert?

I'd follow up with your requirements accordingly.

@komalharshita komalharshita added gssoc:invalid This doesn't seem right and removed type:accessibility ui labels May 25, 2026
@komalharshita

Copy link
Copy Markdown
Owner

Closing this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gssoc:invalid This doesn't seem right gssoc-2026

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UI improvisation

3 participants